Skip to content

Conversation

benjeffery
Copy link
Contributor

Recent dependancy updates (specifically click as shown in #374) are no longer supporting 3.9. To avoid pinning or writing code to deal with multiple click versions, it is probably time to drop 3.9.

@benjeffery benjeffery force-pushed the drop3.9 branch 2 times, most recently from 010de2d to 685f821 Compare May 12, 2025 09:10
@coveralls
Copy link
Collaborator

coveralls commented May 12, 2025

Coverage Status

coverage: 98.137% (-0.005%) from 98.142%
when pulling 6d4c703 on benjeffery:drop3.9
into 6708eb1 on sgkit-dev:main.

@benjeffery benjeffery marked this pull request as ready for review May 12, 2025 09:14
@benjeffery
Copy link
Contributor Author

Looks like ruff has peppered the code with strict=False on every zip. I'll turn that off.

@jeromekelleher
Copy link
Contributor

I'd like to turn off as much of the Ruff stuff as we can. There's a difference between not supporting an old version and gratuitously breaking the code on old versions for the sake of making something that's of limited value (type annotations; IMO) slightly nicer looking.

@jeromekelleher
Copy link
Contributor

Maybe that's not feasible - let's not worry too much about it. I do resent the time spent on managing type annotations...

@jeromekelleher jeromekelleher linked an issue May 12, 2025 that may be closed by this pull request
@jeromekelleher
Copy link
Contributor

Looks like this pulled in some tskit stuff?

@benjeffery
Copy link
Contributor Author

I've moved the pin to the dev requirements as the changes don't break the non-CI code.
tskit stuff removed.

@jeromekelleher jeromekelleher merged commit 6680178 into sgkit-dev:main May 12, 2025
13 of 14 checks passed
@benjeffery benjeffery deleted the drop3.9 branch May 12, 2025 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

vcf2zarr leaks semaphore objects
3 participants